Skip to content

fix: strip connection-nominated proxy headers - #217

Merged
maralbahari merged 1 commit into
vllm-project:mainfrom
StevenWang-CY:fix-proxy-connection-options
Aug 28, 2026
Merged

fix: strip connection-nominated proxy headers#217
maralbahari merged 1 commit into
vllm-project:mainfrom
StevenWang-CY:fix-proxy-connection-options

Conversation

@StevenWang-CY

Copy link
Copy Markdown
Contributor

Summary

RFC 9110 §7.6.1 requires intermediaries to consume the Connection field and remove every header it names before forwarding a message. The proxy removed the standard hop-by-hop fields, while extension fields named by Connection remained in forwarded requests and responses.

Collect option names from every Connection field value and apply them during request and response filtering. Parsing handles comma-separated values, repeated fields, case-insensitive header names, and optional whitespace while preserving unrelated end-to-end headers.

Test Plan

  • cargo test -p agentic-server-core proxy::tests
  • cargo test --workspace
  • cargo clippy --all-targets -- -D warnings
  • cargo fmt --all -- --check
  • uvx pre-commit==4.4.0 run --all-files

Signed-off-by: StevenWang-CY <203932027+StevenWang-CY@users.noreply.github.com>
@maralbahari
maralbahari merged commit 04ec1a0 into vllm-project:main Aug 28, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants